Skip to content

Conversation

@yamt
Copy link
Collaborator

@yamt yamt commented Sep 4, 2025

This is a preparation for
#4364

@yamt
Copy link
Collaborator Author

yamt commented Sep 4, 2025

can anyone familiar with the platform test/review?
i haven't tested this at all.

@yamt yamt changed the title product-mini/platforms/rt-thread: migrate to wasm_runtime_instantiate… product-mini/platforms/rt-thread: migrate to InstantiationArgs2 Sep 4, 2025
@yamt
Copy link
Collaborator Author

yamt commented Sep 4, 2025

@yamt
Copy link
Collaborator Author

yamt commented Sep 4, 2025

@lum1n0us
Copy link
Collaborator

lum1n0us commented Sep 5, 2025

can anyone familiar with the platform test/review? i haven't tested this at all.

code, LGTM. But no test platform.

@lum1n0us lum1n0us added the enhancement Check if this issue/PR enhances a feature; scripts will use this info. label Sep 9, 2025
Copy link
Collaborator

@lum1n0us lum1n0us left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. But shall we merge it without CI?

@yamt
Copy link
Collaborator Author

yamt commented Sep 11, 2025

@dongsheng28849455 are you still interested in rt-thread support?

@lum1n0us lum1n0us merged commit 75e5b4e into bytecodealliance:main Oct 23, 2025
588 of 590 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement Check if this issue/PR enhances a feature; scripts will use this info.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants